Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Experiment adding a dynamic canonical URL to mkdocs #3

Draft
wants to merge 5 commits into
base: main
Choose a base branch
from

Conversation

benjaoming
Copy link
Collaborator

@benjaoming benjaoming commented Dec 7, 2022

This PR is currently stuck, looking for a way to use !ENV in .mkdocs

@benjaoming benjaoming force-pushed the canonical-urls branch 3 times, most recently from de0e4c0 to e9fecf4 Compare December 7, 2022 11:19
@benjaoming
Copy link
Collaborator Author

This doesn't work because of yaml_load_safely

site_url is set to null:

 cat mkdocs.yml
docs_dir: docs
extra_css:
- /_/static/css/badge_only.css
- /_/static/css/readthedocs-doc-embed.css
extra_javascript:
- readthedocs-data.js
- /_/static/core/js/readthedocs-doc-embed.js
- /_/static/javascript/readthedocs-analytics.js
google_analytics: null
markdown_extensions:
- markdown_include.include:
    base_path: .
- admonition
plugins:
- search
- mkdocstrings:
    handlers:
      python:
        options:
          docstring_style: sphinx
site_name: Basic MkDocs Example Project
site_url: null
theme:
  highlightjs: true
  name: readthedocs 

@benjaoming benjaoming marked this pull request as draft December 7, 2022 12:44
@benjaoming
Copy link
Collaborator Author

Blocked by readthedocs/readthedocs.org#8529

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant